Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix bug where we could fail to emit user requested secrets or configmaps #3925

Merged
merged 1 commit into from
Apr 11, 2024

Conversation

matthchr
Copy link
Member

@matthchr matthchr commented Apr 8, 2024

If the user requests a particular secret or configmap value be written, we should error if we are unable to write it and retry.

If applicable:

  • this PR contains documentation
  • this PR contains tests
  • this PR contains YAML Samples

@matthchr matthchr added bug 🪲 Something isn't working breaking A change which likely be breaking labels Apr 8, 2024
@matthchr matthchr added this to the v2.7.0 milestone Apr 8, 2024
@matthchr matthchr self-assigned this Apr 8, 2024
@matthchr
Copy link
Member Author

matthchr commented Apr 9, 2024

/ok-to-test sha=ca1ba70

@matthchr matthchr force-pushed the feature/fix-skip-bug branch from ca1ba70 to 9f35693 Compare April 10, 2024 23:35
If the user requests a particular secret or configmap value be written,
we should error if we are unable to write it and retry.
@matthchr matthchr force-pushed the feature/fix-skip-bug branch from 9f35693 to fed84ac Compare April 11, 2024 16:38
@matthchr
Copy link
Member Author

/ok-to-test sha=fed84ac

@matthchr matthchr added this pull request to the merge queue Apr 11, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 11, 2024
@matthchr matthchr added this pull request to the merge queue Apr 11, 2024
Merged via the queue into Azure:main with commit a6ff798 Apr 11, 2024
8 checks passed
@matthchr matthchr deleted the feature/fix-skip-bug branch April 11, 2024 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking A change which likely be breaking bug 🪲 Something isn't working
Projects
Development

Successfully merging this pull request may close these issues.

3 participants